Skip to content

Comments

Смирнов Алексей#36

Open
AlexSm13 wants to merge 8 commits intourfu-2016:masterfrom
AlexSm13:master
Open

Смирнов Алексей#36
AlexSm13 wants to merge 8 commits intourfu-2016:masterfrom
AlexSm13:master

Conversation

@AlexSm13
Copy link

@AlexSm13 AlexSm13 commented Oct 30, 2016

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍏 Пройден линтинг и базовые тесты

@FrodoTheTrue
Copy link

В целом ок. Хочется еще увидеть:

  1. Картиночек среди колонок
  2. Вертикальный текст
  3. Будет круто если подключишь шрифты не только с google fonts

@FrodoTheTrue
Copy link

🍅

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍏 Пройден линтинг и базовые тесты


.post-2
{
column-count: 3;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

безвендорное правило должно идти после

@FrodoTheTrue
Copy link

Стало лучше, хочется еще увидеть картинку на несколько стобцов

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍏 Пройден линтинг и базовые тесты

@FrodoTheTrue
Copy link

FrodoTheTrue commented Nov 2, 2016

В мозиле совсем разломалось
2016-11-02 23 03 16
В этом браузере есть некоторые проблемы, но верстка не должна так сильно разваливаться и текст должен быть читаемым

<header>
<h1 class="headline">USA TODAY</h1>
<aside class="weather"><span class="vertical">Weather</span>
<img src="image/d.sun.png" alt="sun">+12</aside>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

не хватает еще title

@FrodoTheTrue
Copy link

🍅

height: 140px;
}

div

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

привязываться к тегам обычно плохая практика, а к диву особенно нехорошо

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants